Skip to content

Vulkan: Fix HDR pipeline init to revert back to sdr_render_pass#18680

Merged
LibretroAdmin merged 1 commit intolibretro:masterfrom
MajorPainTheCactus:vulkan-hdr-init-cleanup
Feb 4, 2026
Merged

Vulkan: Fix HDR pipeline init to revert back to sdr_render_pass#18680
LibretroAdmin merged 1 commit intolibretro:masterfrom
MajorPainTheCactus:vulkan-hdr-init-cleanup

Conversation

@MajorPainTheCactus
Copy link
Copy Markdown
Contributor

@MajorPainTheCactus MajorPainTheCactus commented Feb 3, 2026

Summary

  • Use correct render pass: Changes vk->render_pass to vk->sdr_render_pass when restoring pipeline state after the HDR tonemapping readback pipeline setup, to correctly reference the SDR render pass

Big thanks to @warmenhoven who found this fix and did the work in tracking it down!

Fix indentation of the HDR pipeline initialization block inside
vulkan_init_pipelines to use consistent 3-space indentation matching
the surrounding code style.

Use vk->sdr_render_pass instead of vk->render_pass when restoring
pipeline state after the HDR tonemapping readback pipeline setup,
to correctly reference the SDR render pass.
@MajorPainTheCactus MajorPainTheCactus changed the title Vulkan: Fix HDR pipeline init indentation and use sdr_render_pass Vulkan: Fix HDR pipeline init use sdr_render_pass Feb 3, 2026
@MajorPainTheCactus MajorPainTheCactus changed the title Vulkan: Fix HDR pipeline init use sdr_render_pass Vulkan: Fix HDR pipeline init to revert back to sdr_render_pass Feb 3, 2026
@LibretroAdmin LibretroAdmin merged commit 256f88c into libretro:master Feb 4, 2026
35 checks passed
@MajorPainTheCactus MajorPainTheCactus deleted the vulkan-hdr-init-cleanup branch February 16, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants